fix(about_py): correct PyTorch and LaTeX capitalization - #124
Merged
Conversation
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
✅ Deploy Preview for majestic-griffin-10b166 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes two capitalization typos in the Persian translation of the “About Python” lecture, aligning common library/format names with their standard branding.
Changes:
- Correct
Pytorch→PyTorchin the NumPy alternatives section. - Correct
LaTex→LaTeXin the Matplotlib description.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mmcky
added a commit
that referenced
this pull request
Aug 20, 2026
…160) Recovers the sync failures recorded as lecture-python-programming#610 and #614, without re-translating anything. Source PR #595 changed exactly two lines of about_py.md — the capitalisation `Pytorch` -> `PyTorch` in two places. That fix is already present here, applied by hand in #124 on 2026-06-19: the target carries `PyTorch` three times and `Pytorch` zero times, matching source. The source has not moved on this lecture since #595, so only the state file was stale. A `translate forward` regeneration was attempted first and deliberately discarded. It rewrote 266 lines to deliver a two-character casing fix that was already applied, and it dropped translator content — the tool's own triage flagged an added PyTorch/MATLAB sentence and a Java code comment as "TARGET has additions that were lost during resync", and `git log` traces that content to Zahra's 2025-10-30 commit (#14). Per the 2026-08-20 ruling on QuantEcon/action-translation#276, a natively-translated lecture is not regenerated when a deterministic repair will do. `model`, `tool-version` and `section-count` are deliberately left recording how the text was actually produced, since nothing was regenerated. Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Flagged by Copilot in #121 review. Both are pre-existing typos in the Persian translation, not regressions from any translation-sync PR.
Test plan
🤖 Generated with Claude Code